Skip to content

Conversation

@tmssngr
Copy link
Contributor

@tmssngr tmssngr commented Oct 1, 2025

Control: don't send MouseMove events with stateMask set for BUTTON1 before the MouseDown event was sent

These events are already added to the dragDetectionQueue, so there is no need to send/post them now.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 1, 2025

Test Results

  115 files   -  3    115 suites   - 3   11m 20s ⏱️ - 3m 59s
4 614 tests  - 37  4 598 ✅  - 36  16 💤  - 1  0 ❌ ±0 
  312 runs   - 18    309 ✅  - 17   3 💤  - 1  0 ❌ ±0 

Results for commit 77b255c. ± Comparison against base commit a9598b8.

This pull request removes 37 tests.
AllGTKTests Test_GtkConverter ‑ test_HeuristicASCII_dollarSign
AllGTKTests Test_GtkConverter ‑ test_HeuristicASCII_emptyString
AllGTKTests Test_GtkConverter ‑ test_HeuristicASCII_letterA
AllGTKTests Test_GtkConverter ‑ test_HeuristicASCII_letters
AllGTKTests Test_GtkConverter ‑ test_HeuristicUTF16LE_null
AllGTKTests Test_GtkConverter ‑ test_HeuristicUTF16_AsciiLetters
AllGTKTests Test_GtkConverter ‑ test_HeuristicUTF16_Asciiletter
AllGTKTests Test_GtkConverter ‑ test_HeuristicUTF16_LotsOfLetters
AllGTKTests Test_GtkConverter ‑ test_HeuristicUTF16_letter
AllGTKTests Test_GtkConverter ‑ test_HeuristicUTF16_letters
…
This pull request skips 1 test.
org.eclipse.swt.tests.junit.Test_org_eclipse_swt_widgets_Shell ‑ test_activateEventSend

♻️ This comment has been updated with latest results.

@akurtakov akurtakov force-pushed the 2119-fix-mouse-event-order branch from a90eda9 to 1f4d1c0 Compare October 17, 2025 15:47
@tmssngr tmssngr force-pushed the 2119-fix-mouse-event-order branch 2 times, most recently from 366ee0e to 77b255c Compare November 5, 2025 09:59
…atform#2119

Control: don't send MouseMove events with stateMask set for BUTTON1
before the MouseDown event was sent

These events are already added to the dragDetectionQueue, so there is no
need to send/post them now.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant